Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add a Dockerfile #3

Open
wants to merge 8 commits into
base: master
Choose a base branch
from
Open

add a Dockerfile #3

wants to merge 8 commits into from

Conversation

jcupitt
Copy link

@jcupitt jcupitt commented Jan 25, 2018

It seems to work for me ... test with:

docker build \
	--build-arg VCS_REF=`git rev-parse --short HEAD` \
	-t <user>/structural-pipeline:latest .

Fetching the FSL key seems a little unreliable, I'm not sure why. If you
get an error, try just running again.

see #2

It seems to work for me ... test with:

	docker build \
		--build-arg VCS_REF=`git rev-parse --short HEAD` \
		-t <user>/structural-pipeline:latest .

Fetching the FSL key seems a little unreliable, I'm not sure why. If you
get an error, try just running again.

see #2
@jcupitt
Copy link
Author

jcupitt commented Jan 25, 2018

(Dockerfile shamelessly copied from MIRTK)

@schuhschuh
Copy link

Sweet! BTW I myself have no permissions to merge. Either wait for @amakropoulos's reply or get permissions from in-house if your part of BioMedIA/dHCP.

I made it urge people to do a build in docker, perhaps that's not wise
@jcupitt
Copy link
Author

jcupitt commented Jan 25, 2018

I added some notes to the README. It should probably use -v to attach the user's data files to the container for processing, I'll experiment.

fsl5 has "fsl5.0-" in front of all exe names

the dockerfile needs to get root shell to run the fsl init script
@jcupitt
Copy link
Author

jcupitt commented Feb 2, 2018

It seems to be working well now. I'm going to merge to master on my fork, I'll leave this here.

The README has some notes -- tldr: build the pipeline to a docker image with:

# docker build -t <user>/structural-pipeline:latest .

Run the pipeline on the file sub-CC00183XX11_ses-60300_T2w.nii.gz in the subdirectory data of the current directory:

# docker run --rm -t -v $PWD/data:/data \
    <user>/structural-pipeline:latest \
    bash -c ". /etc/fsl/fsl.sh; \
        cd /usr/src/structural-pipeline; \
        ./dhcp-pipeline.sh subject1 session1 44 \
            -d /data -T2 /data/sub-CC00183XX11_ses-60300_T2w.nii.gz -t 8"

I'll try and get OpenMOLE using this next.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants